Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tweak wording in using-notebooks #193

Merged
merged 1 commit into from
Dec 3, 2023
Merged

Tweak wording in using-notebooks #193

merged 1 commit into from
Dec 3, 2023

Conversation

EliahKagan
Copy link
Owner

Adding raises-exception support may be out of scope for vscode-jupyter due to lack of widespread interest in the feature. This slightly rewords the second on VS Code to avoid falsely implying that the project has committed to adding raises-exception support or that it will necessarily be added.

Further refinement may be possible in the future, but I'm doing this small rewording now to avoid potentially confusing inaccuracy.

Adding raises-exception support may be out of scope for
vscode-jupyter due to lack of widespread interest in the feature.
This slightly rewords the second on VS Code to avoid falsely
implying that the project has committed to adding raises-exception
support or that it will necessarily be added.

Further refinement may be possible in the future, but I'm doing
this small rewording now to avoid potentially confusing inaccuracy.
@EliahKagan EliahKagan merged commit c01eedc into main Dec 3, 2023
91 checks passed
@EliahKagan EliahKagan deleted the ek/docs branch December 3, 2023 23:42
EliahKagan added a commit that referenced this pull request Dec 4, 2023
EliahKagan added a commit that referenced this pull request Dec 4, 2023
It looks like the feature is being added to vscode-jupyter! This
reverts the wording since the old wording turns out more accurate,
but I'll want to update this again once I've checked the details
and (if ready) tried it out on our notebooks with raises-exception.

This reverts commit c01eedc.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant